-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PowerPages] [Create-Site] Implement multi-step site creation with enhanced input handling #1063
Merged
amitjoshi438
merged 24 commits into
main
from
users/amitjoshi/copilotSiteCreateQuickPick
Dec 16, 2024
Merged
[PowerPages] [Create-Site] Implement multi-step site creation with enhanced input handling #1063
amitjoshi438
merged 24 commits into
main
from
users/amitjoshi/copilotSiteCreateQuickPick
Dec 16, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…lyFileSystemProvider for site page previews
… CreateSiteHelper to utilize it
…omponents for site creation functionality
…ions and improve readability; add CreateSiteTypes for better type management
…ors in previewSitePagesContent function
…in EditableFileSystemProvider
… user inputs in CreateSiteHelper
…ntOptions interface
amitjoshi438
changed the title
Implement multi-step site creation with enhanced input handling
[PowerPages] [Create-Site] Implement multi-step site creation with enhanced input handling
Nov 28, 2024
priyanshu92
reviewed
Dec 2, 2024
src/common/chat-participants/powerpages/commands/create-site/CreateSiteTypes.ts
Outdated
Show resolved
Hide resolved
src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts
Outdated
Show resolved
Hide resolved
src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts
Outdated
Show resolved
Hide resolved
…reateSiteTypes.ts Co-authored-by: Priyanshu Agrawal <[email protected]>
…reateSiteHelper.ts Co-authored-by: Priyanshu Agrawal <[email protected]>
…reateSiteHelper.ts Co-authored-by: Priyanshu Agrawal <[email protected]>
priyanshu92
approved these changes
Dec 3, 2024
sanxsth
approved these changes
Dec 4, 2024
src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts
Show resolved
Hide resolved
…github.com/microsoft/powerplatform-vscode into users/amitjoshi/copilotSiteCreateQuickPick
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes several changes to the
PowerPagesChatParticipant
and related files to enhance functionality and improve error handling. The most important changes involve adding new commands for site creation, updating endpoint handling, and improving utility functions.Enhancements to site creation functionality:
src/common/chat-participants/powerpages/PowerPagesChatParticipant.ts
: AddedregisterButtonCommands
to initialize button commands for site creation. [1] [2]src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts
: Added functionscollectSiteCreationInputs
andgetUpdatedPageContent
to gather inputs for site creation and update page content. [1] [2]Improved endpoint handling:
src/common/chat-participants/powerpages/PowerPagesChatParticipant.ts
: Added a check for the availability of the intelligence API endpoint before proceeding with operations.src/common/services/ArtemisService.ts
: IncludedendpointStamp
in the response fromgetIntelligenceEndpoint
. [1] [2]Utility function updates:
src/common/utilities/Utils.ts
: ModifiedgetEnvList
to handle cases whereendpointStamp
is undefined and to return environment information in a standardized format. [1] [2]Additional changes:
src/common/chat-participants/powerpages/commands/create-site/CreateSiteTypes.ts
: AddedISiteInputState
interface to store site creation input state.src/common/constants.ts
: AddedIEnvInfo
interface to represent environment information.